-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix minor ledger reported issues #107
Fix minor ledger reported issues #107
Conversation
ee64404
to
316208b
Compare
5bc1ddc
to
316208b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a test to check microtez to string conversion with a fractional microtez value with large trailing zeros. A value with no trailing zeros like 19.999999 tez.
|
ef7e422
to
51dced4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ledger report some small issues:
Done in Fix minor issues like NULL pointer check #108
Makefile: remove silenced warning
src/to_string.c:344
.to_string: improve the copy_string documentation
For loop variable changed in the body
insrc/to_string:279
microtez_to_string: clarify the trailing 0s elimination
Long switch case
insrc/operation.c:355-491
Postponed for another MR: A CodeQL identifies
Long switch case
insrc/operation.c:355-491
#109